Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Import Gradle project via Buildship if Gradle Build Server is not available #3245

Merged
merged 2 commits into from
Sep 5, 2024

Conversation

jdneo
Copy link
Contributor

@jdneo jdneo commented Aug 21, 2024

There will be the case when user has imported the gradle project by build server. Then, the user disable/uninstall the gradle extension.

In this case, the build server project should not be excluded when gradle importer scans the folder to import.

But, so far I cannot figure out a perfect way to do that. The nature id for build server is hard coded. @snjeza @rgrunber Do you know if we can have better approach than this?

@snjeza
Copy link
Contributor

snjeza commented Aug 21, 2024

But, so far I cannot figure out a perfect way to do that. The nature id for build server is hard coded

It works fine for me. Nature ids are always hard coded. Could you, please, add a test?

@jdneo
Copy link
Contributor Author

jdneo commented Aug 22, 2024

It works fine for me. Nature ids are always hard coded. Could you, please, add a test?

Thank you! if it looks fine, I'll continue the work.

- Due to the build server has a higher priority, if the Buildship
  kicks in during import. This mean build server fails to import
  the projects and Buildship will be used and the footprint of the
  build server will be removed if exists.

Signed-off-by: Sheng Chen <[email protected]>
Signed-off-by: Sheng Chen <[email protected]>
@jdneo
Copy link
Contributor Author

jdneo commented Sep 2, 2024

test this please

@jdneo jdneo marked this pull request as ready for review September 2, 2024 12:09
@snjeza
Copy link
Contributor

snjeza commented Sep 2, 2024

test this please

Copy link
Contributor

@snjeza snjeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jdneo jdneo merged commit da3076e into eclipse-jdtls:master Sep 5, 2024
5 of 6 checks passed
@jdneo jdneo deleted the cs/gradle-remove branch September 5, 2024 01:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants